cssmatcher: Remove type argument from name matcher
authorBenjamin Otte <otte@redhat.com>
Fri, 5 Feb 2016 22:43:05 +0000 (23:43 +0100)
committerBenjamin Otte <otte@redhat.com>
Fri, 5 Feb 2016 22:43:05 +0000 (23:43 +0100)
We no longer use types when matching names.

gtk/gtkcssmatcher.c
gtk/gtkcssmatcherprivate.h
gtk/gtkcssselector.c

index 8dd26d52fe2566d288fd602cb677179b8cab03a9..8afdab1b26adc07fc9f6f777384913f070807960 100644 (file)
@@ -74,8 +74,7 @@ gtk_css_matcher_widget_path_get_state (const GtkCssMatcher *matcher)
 
 static gboolean
 gtk_css_matcher_widget_path_has_name (const GtkCssMatcher     *matcher,
-                                      /*interned*/ const char *name,
-                                      GType                    type)
+                                      /*interned*/ const char *name)
 {
   const GtkWidgetPath *siblings;
 
@@ -85,11 +84,7 @@ gtk_css_matcher_widget_path_has_name (const GtkCssMatcher     *matcher,
       const char *path_name = gtk_widget_path_iter_get_object_name (siblings, matcher->path.sibling_index);
 
       if (path_name == NULL)
-        {
-          if (type == 0)
-            return FALSE;
-          return g_type_is_a (gtk_widget_path_iter_get_object_type (siblings, matcher->path.sibling_index), type);
-        }
+        path_name = g_type_name (gtk_widget_path_iter_get_object_type (siblings, matcher->path.sibling_index));
 
       return path_name == name;
     }
@@ -98,11 +93,7 @@ gtk_css_matcher_widget_path_has_name (const GtkCssMatcher     *matcher,
       const char *path_name = gtk_widget_path_iter_get_object_name (matcher->path.path, matcher->path.index);
 
       if (path_name == NULL)
-        {
-          if (type == 0)
-            return FALSE;
-          return g_type_is_a (gtk_widget_path_iter_get_object_type (matcher->path.path, matcher->path.index), type);
-        }
+        path_name = g_type_name (gtk_widget_path_iter_get_object_type (matcher->path.path, matcher->path.index));
 
       return path_name == name;
     }
@@ -251,15 +242,9 @@ gtk_css_matcher_node_get_state (const GtkCssMatcher *matcher)
 
 static gboolean
 gtk_css_matcher_node_has_name (const GtkCssMatcher     *matcher,
-                               /*interned*/ const char *name,
-                               GType                    type)
+                               /*interned*/ const char *name)
 {
-  const char *node_name = gtk_css_node_get_name (matcher->node.node);
-
-  if (node_name == NULL)
-    return g_type_is_a (gtk_css_node_get_widget_type (matcher->node.node), type);
-  
-  return node_name == name;
+  return gtk_css_node_get_name (matcher->node.node) == name;
 }
 
 static gboolean
@@ -375,8 +360,7 @@ gtk_css_matcher_any_get_state (const GtkCssMatcher *matcher)
 
 static gboolean
 gtk_css_matcher_any_has_name (const GtkCssMatcher     *matcher,
-                              /*interned*/ const char *name,
-                              GType                    type)
+                              /*interned*/ const char *name)
 {
   return TRUE;
 }
@@ -457,11 +441,10 @@ gtk_css_matcher_superset_get_state (const GtkCssMatcher *matcher)
 
 static gboolean
 gtk_css_matcher_superset_has_name (const GtkCssMatcher     *matcher,
-                                   /*interned*/ const char *name,
-                                   GType                    type)
+                                   /*interned*/ const char *name)
 {
   if (matcher->superset.relevant & GTK_CSS_CHANGE_NAME)
-    return _gtk_css_matcher_has_name (matcher->superset.subset, name, type);
+    return _gtk_css_matcher_has_name (matcher->superset.subset, name);
   else
     return TRUE;
 }
index 82be76228abd0b3a3673de5a21a99d9f62680aeb..29890a84c566a93749ce29a6afc5a32581bde10d 100644 (file)
@@ -37,8 +37,7 @@ struct _GtkCssMatcherClass {
 
   GtkStateFlags   (* get_state)                   (const GtkCssMatcher   *matcher);
   gboolean        (* has_name)                    (const GtkCssMatcher   *matcher,
-                                                   /*interned*/const char*name,
-                                                   GType                  type);
+                                                   /*interned*/const char*name);
   gboolean        (* has_class)                   (const GtkCssMatcher   *matcher,
                                                    GQuark                 class_name);
   gboolean        (* has_id)                      (const GtkCssMatcher   *matcher,
@@ -109,10 +108,9 @@ _gtk_css_matcher_get_state (const GtkCssMatcher *matcher)
 
 static inline gboolean
 _gtk_css_matcher_has_name (const GtkCssMatcher     *matcher,
-                           /*interned*/ const char *name,
-                           GType                    type)
+                           /*interned*/ const char *name)
 {
-  return matcher->klass->has_name (matcher, name, type);
+  return matcher->klass->has_name (matcher, name);
 }
 
 static inline gboolean
index b23e1fdc1c2585d66bb025112f6014d54f4e5b19..e40a8df64e5ad322c91e78af4de09d2ae7203d4e 100644 (file)
@@ -633,7 +633,7 @@ static gboolean
 match_name (const GtkCssSelector *selector,
             const GtkCssMatcher  *matcher)
 {
-  return _gtk_css_matcher_has_name (matcher, selector->name.reference->name, selector->name.reference->type);
+  return _gtk_css_matcher_has_name (matcher, selector->name.reference->name);
 }
 
 static guint